NetNews Offline 2/NetNews Offline Volume 2.iso/news/comp/lang/c-part1/6856

Text File | 1996-08-05 | 1.8 KB | 49 lines

Path: sun001.spd.dsccc.com!spd!jmccarty
From: [e-mail] (Mike McCarty)
Newsgroups: comp.lang.c
Subject: Re: Watch out!  C "gotcha!"
Date: 15 Feb 1996 19:38:01 GMT
Organization: DSC Communications Corporation, Plano, Texas USA
Message-ID: <4g022p$[e-mail]>
References: <4fthhh$[e-mail]> <[e-mail]>
NNTP-Posting-Host: aplo139.spd.dsccc.com

In article <[e-mail]>, Larry Weiss  <[e-mail]> wrote:
)Michael Schwarz wrote:
) >
) > Many of you may already know this, but even after ten years of coding
) > in C, this took my by surprise at first.  A co-worker and I were looking
) > at some library C code where I work and we noticed that the "default"
) > clause of one of the switches was misspelled as "defalt."
) > ...
) > The thing that scares me the most about this is that either I've never before
) > made this particular mistake or for the last ten years I've been leaving
) > behind a number of ticking code bombs...
) >
) > I invite your comments on this (and your knowledge -- I'm a good C programmer
) > but I don't write compilers and I'm not fully conversant with the full
) > ANSI spec on C).  Please feel free to e-mail me.
) >
)
)
)It's a good observation.   I'd suggest a
)
)    grep "t:" *.c | grep -v "default"
)
)or equivalent inspection of any body of C source every once in a while.


Perhaps just ":" would be better. But of course, someone, somewhere, is
putting the colons on the next line.

The really only truly way to watch for this is to use a language which
forces declaration of labels. And does =not= use a label construct for
multiway jump or case statement (C does not have a true case statment,
it has a multiway jump).

Mike
----
char *p="char *p=%c%s%c;main(){printf(p,34,p,34);}";main(){printf(p,34,p,34);}

I don't speak for DSC.         <- They make me say that.